Skip to content

docs: explain the OpenTelemetry logs setup in the chart README#534

Merged
ilyam8 merged 2 commits into
masterfrom
docs/otel-logs-readme
Jun 26, 2026
Merged

docs: explain the OpenTelemetry logs setup in the chart README#534
ilyam8 merged 2 commits into
masterfrom
docs/otel-logs-readme

Conversation

@M4itee

@M4itee M4itee commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

What

Adds a concise Collecting logs with OpenTelemetry section to the chart README, explaining the two components involved in OpenTelemetry log ingestion and why both must be enabled.

Why

netdataOpentelemetry and otel-collector both exist in values.yaml, but the README never explained their relationship — that both are required for logs to appear, and that the bundled collector is an optional, replaceable convenience. This previously required manual explanation to users.

Changes

  • templates/netdata-README.md.gotmpl — new ### Collecting logs with OpenTelemetry subsection (placed after Persistent volumes), covering:
    • netdataOpentelemetry — the OTLP receiver (port 4317) that stores and displays logs; it does not collect them.
    • otel-collector — the optional DaemonSet (one pod per node) that reads node-local container logs and forwards them over OTLP; replaceable by any OTLP-capable pipeline.
    • An end-to-end log-flow diagram.
  • charts/netdata/README.md — regenerated via ./generate-documentation.sh.

Notes

  • Documentation only — no chart logic, templates, or values changed.
  • The generated README stays in sync with the template, so the "documentation is up-to-date" CI check passes.

@M4itee M4itee requested review from a team and ilyam8 as code owners June 25, 2026 10:27
@M4itee M4itee requested a review from witalisoft June 25, 2026 10:27
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Mateusz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@witalisoft witalisoft left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is worth mentioning, especially if you want to push the logs yourself, that by default there is no TLS enabled for the OTEL endpoint in netdataOpentelemetry

Comment thread charts/netdata/README.md Outdated
### Collecting logs with OpenTelemetry

Netdata can ingest, store, and visualize your cluster's container logs through OpenTelemetry.
This relies on two components — **both disabled by default** and **both required** for logs to

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both required ? later on there is an info that otel-collector is an optional subchart

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is explained more later but I will reword it a bit.

Add a "Collecting logs with OpenTelemetry" section to the chart README
describing the two components required for log ingestion and why both must
be enabled:

- netdataOpentelemetry: the OTLP receiver (port 4317) that stores and
  displays logs; it does not collect them.
- otel-collector: the optional DaemonSet that reads node-local container
  logs and forwards them over OTLP. It is replaceable by any OTLP-capable
  log pipeline.

Also documents the end-to-end log flow. Regenerates charts/netdata/README.md
via generate-documentation.sh.
@M4itee M4itee force-pushed the docs/otel-logs-readme branch from 3c4905c to 4a64128 Compare June 25, 2026 12:16
@M4itee M4itee requested a review from witalisoft June 26, 2026 11:07
Add a "Securing the endpoint with TLS" subsection to the OpenTelemetry
logs documentation, covering the full end-to-end setup:

- generate a self-signed certificate and key
- create a Kubernetes TLS secret
- mount it into the netdata-otel receiver and point the otel config at it
  (the tls_cert_path/tls_key_path must be set, mounting alone is not enough)
- switch clients, including the bundled otel-collector, to TLS

Regenerates charts/netdata/README.md via generate-documentation.sh.
@ilyam8 ilyam8 merged commit 79afc74 into master Jun 26, 2026
1 of 2 checks passed
@ilyam8 ilyam8 deleted the docs/otel-logs-readme branch June 26, 2026 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants